out of date comment in commit_ts.c

  • Jump to comment-1
    nathandbossart@gmail.com2022-07-26T17:33:43+00:00
    Hi hackers, I noticed that commit_ts.c has the following comment: * XLOG interactions: this module generates an XLOG record whenever a new * CommitTs page is initialized to zeroes. Also, one XLOG record is * generated for setting of values when the caller requests it; this allows * us to support values coming from places other than transaction commit. * Other writes of CommitTS come from recording of transaction commit in * xact.c, which generates its own XLOG records for these events and will * re-perform the status update on redo; so we need make no additional XLOG * entry here. IIUC the ability for callers to request WAL record generation is no longer possible as of 08aa89b [0]. Should the second sentence be removed? [0] https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=08aa89b -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
    • Jump to comment-1
      nathandbossart@gmail.com2022-07-27T20:29:52+00:00
      On Tue, Jul 26, 2022 at 10:33:43AM -0700, Nathan Bossart wrote: > IIUC the ability for callers to request WAL record generation is no longer > possible as of 08aa89b [0]. Should the second sentence be removed? Here's a patch. -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
      • Jump to comment-1
        thomas.munro@gmail.com2022-08-09T01:02:15+00:00
        On Thu, Jul 28, 2022 at 8:30 AM Nathan Bossart <nathandbossart@gmail.com> wrote: > On Tue, Jul 26, 2022 at 10:33:43AM -0700, Nathan Bossart wrote: > > IIUC the ability for callers to request WAL record generation is no longer > > possible as of 08aa89b [0]. Should the second sentence be removed? > > Here's a patch. Pushed.
        • Jump to comment-1
          nathandbossart@gmail.com2022-08-09T03:29:15+00:00
          On Tue, Aug 09, 2022 at 01:02:15PM +1200, Thomas Munro wrote: > Pushed. Thanks! -- Nathan Bossart Amazon Web Services: https://aws.amazon.com